Skip to content

Conversation

@lz-bro
Copy link
Contributor

@lz-bro lz-bro commented Apr 28, 2025

#617 has a pipeline failure for Ubuntu 20.04 has retired.

@lz-bro lz-bro force-pushed the debug-test-ubuntu-latest branch 2 times, most recently from 8ecc1ad to 30d695c Compare April 28, 2025 03:23
@lz-bro lz-bro force-pushed the debug-test-ubuntu-latest branch from 30d695c to bab737b Compare April 28, 2025 03:31
Copy link
Collaborator

@en-sc en-sc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

test:
name: Test debug (Ubuntu)
runs-on: ubuntu-20.04
runs-on: ubuntu-latest
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alternatively, Ubuntu 24.04 can be used. However, AFAIU, GitHub runner supports the two latest LTS versions of Ubuntu, so I'm also leaning towards using ubuntu-latest.
FIY: actions/runner-images#11101

run: |
sudo apt-get update
sudo apt-get install -y device-tree-compiler build-essential
sudo apt-get install -y device-tree-compiler build-essential libjim-dev
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a clarification:
libjim-dev is the new requirement of OpenOCD. Previously it was build in-tree.

@en-sc
Copy link
Collaborator

en-sc commented Apr 28, 2025

@aswaterman, can you please take a look at this one?
This is quite urgent, since the CI is broken right now.

@aswaterman
Copy link
Collaborator

@en-sc Yes, thank you for the ping.

I don't like using ubuntu-latest because we get sudden breakages when github decides to redefine "latest". I'll upgrade to 24.04 instead.

@aswaterman
Copy link
Collaborator

Closed in favor of #619

@aswaterman aswaterman closed this Apr 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants